Skip to content

Ensure DismissButton has a height so it works with iOS VO#5539

Merged
devongovett merged 1 commit into
mainfrom
dismiss-button-height
Dec 7, 2023
Merged

Ensure DismissButton has a height so it works with iOS VO#5539
devongovett merged 1 commit into
mainfrom
dismiss-button-height

Conversation

@devongovett

Copy link
Copy Markdown
Member

Noticed that the dismiss button in popovers was not working on iOS VO on the new homepage. Looks like it was caused by Tailwind's CSS reset which removes the default padding from buttons. That caused it to be zero width/height, which iOS VO will ignore. This adds an inline width and height property to ensure it is always set. Should not affect anything because it's inside a VisuallyHidden already.

@rspbot

rspbot commented Dec 6, 2023

Copy link
Copy Markdown

@rspbot

rspbot commented Dec 6, 2023

Copy link
Copy Markdown
## API Changes

unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any', access: 'private' }
unknown top level export { type: 'any', access: 'private' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'identifier', name: 'Column' }
unknown top level export { type: 'identifier', name: 'Column' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }

@devongovett
devongovett merged commit c465f95 into main Dec 7, 2023
@devongovett
devongovett deleted the dismiss-button-height branch December 7, 2023 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants